Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement sea-ice freeboard cost #821

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

owang01
Copy link
Collaborator

@owang01 owang01 commented Apr 4, 2024

What changes does this PR introduce?

New feature

What is the current behaviour?

Sea-ice freeboard cost is not implemented and therefore sea-ice freeboard measurements cannot be included as part of the cost function to constrain the model.

What is the new behaviour

With this PR, sea-ice freeboard measurements from ICESat-2 can be included as a constrain.

Does this PR introduce a breaking change?

No

Other information:

Suggested addition to tag-index

o pkg/ecco: implement sea-ice freeboard cost

@jm-c jm-c added the adjoint Affects the adjoint model; label triggers full OpenAD test label Apr 4, 2024
@antnguyen13
Copy link
Collaborator

thank you for making this PR @owang01 ! i left a couple of comments in ecco_phys.F. I think it'd be great to leave a reference for how you computed the freeboard, e.g., derived from the equation for "lidar" from this Kwok & Markus, 2018 manuscript https://www.sciencedirect.com/science/article/abs/pii/S0273117717306403?via%3Dihub .

other than this and the potential division by zero--i think there should be a check somewhere in case user set the SEAICE_area_reg = 0. _d 0 ? perhaps there is already such a check in pkg/seaice that i miss?

@antnguyen13
Copy link
Collaborator

also, i didn't check.. but are there different treatments if you're using the various levels of nonlinFreesurf [0-4] ? and/or with/without RFWF?

@jm-c
Copy link
Member

jm-c commented Jul 18, 2024

also, i didn't check.. but are there different treatments if you're using the various levels of nonlinFreesurf [0-4] ? and/or with/without RFWF?

Just to clarify: the Sea-ice freeboard expression here is intended to be compared with obs and is independent of internal model assumptions (including free-surface).

@antnguyen13
Copy link
Collaborator

also, i didn't check.. but are there different treatments if you're using the various levels of nonlinFreesurf [0-4] ? and/or with/without RFWF?

Just to clarify: the Sea-ice freeboard expression here is intended to be compared with obs and is independent of internal model assumptions (including free-surface).

thanks Jean-Michel. Yes, i think you verbally mentioned this at the last (last) devel meeting already, but good to confirm here for the record.

@owang01
Copy link
Collaborator Author

owang01 commented Jul 25, 2024

@antnguyen13 Thank you for your review. I have added comments on freeboard calculation and included Kwok and Markus (2018) as an example reference. I did not add a check for SEAICE_area_reg = 0._d 0, as it was deemed unnecessary based on discussions in one of the devel meetings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adjoint Affects the adjoint model; label triggers full OpenAD test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants